Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @brson (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

The text seemed confusing as-is, since it appeared to have no halting condition (are we iterating over "option"?).

@bors: r+ rollup

I agree, this is nicer, thanks.

📌 Commit b1c5c26 has been approved by steveklabnik

This explanation implies that while let is tied to iterators, which it is not.

Can you recommend an improvement, or does this mean you think it should stay as-is?

Proposal: I should restore the original prose and leave the example code as shown.

@brson: ahhh, shucks, reading this, my brain said "while uses an iterator, sure", but while != while let.

@androm3da how about changing it so that the whole example is visible, and remove the temporary?

@bors: r- for now

@steveklabnik "The temporary" refers to "vec![1, 3, 5, 7, 9, ]"? (sorry, still a bit of a rust newb)

It's all good! I myself made a mistake: when thinking about it, i was thinking this:

let v = vec![1, 3, 5, 7, 9];
while let Some(x) = v.iter() {
    println!("{}", x);
}

but that doesn't work. This should be fine:

let v = vec![1, 3, 5, 7, 9];
let mut iter = v.iter();
while let Some(x) = iter.next() {
    println!("{}", x);
}

Nice! Thank you

@bors r+ 797a0bd

(A comment on the PR is needed to notify people involved, a pushed commit does not notify, so that's the reason for the two day delay).

⌛ Testing commit 797a0bd with merge 3db9bee...

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-debug-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-x-android-t, auto-linux-cross-opt, auto-linux-musl-64-opt, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt, auto-win-msvc-32-opt, auto-win-msvc-64-opt